See example of integration here: kommodity-io/kommodity#69
Currently using our own fork, as created from this PR using replace in go.mod

Hello, thank you for integrating our provider in your project.

It looks like CI is failing, could you please add these changes to your PR?

• Add this line in Dockerfile: COPY pkg/ pkg/
• Add these exclusions in .golangci-lint.yml:

      - linters:
          - dupl
          - lll
        path: pkg/*
  

I will run e2e in #34 as CI secrets are not accessible from forks.

• linters:
  - dupl
  - lll
  path: pkg/*

Done :) Thanks for the quick reply

On second thoughts, moving controllers to pkg/controller is a bit annoying because it "breaks" kubebuilder as it expects controllers to be in internal/controller by default. Also, there may be breaking changes in this package so we don't want other developers to import this package directly.

Would it be OK for you if we create public aliases to export the reconcilers instead? Something like this in pkg/controller/alias.go:

package controller

import "github.com/scaleway/cluster-api-provider-scaleway/internal/controller"

// Following variables provide access to reconcilers implemented in internal/controller.
var (
	NewScalewayClusterReconciler             = controller.NewScalewayClusterReconciler
	NewScalewayMachineReconciler             = controller.NewScalewayMachineReconciler
	NewScalewayManagedClusterReconciler      = controller.NewScalewayManagedClusterReconciler
	NewScalewayManagedControlPlaneReconciler = controller.NewScalewayManagedControlPlaneReconciler
	NewScalewayManagedMachinePoolReconciler  = controller.NewScalewayManagedMachinePoolReconciler
)

The cluster-api project does something similar for their controllers package: https://github.com/kubernetes-sigs/cluster-api/blob/main/controllers/alias.go

On second thoughts, moving controllers to pkg/controller is a bit annoying because it "breaks" kubebuilder as it expects controllers to be in internal/controller by default. Also, there may be breaking changes in this package so we don't want other developers to import this package directly.

Would it be OK for you if we create public aliases to export the reconcilers instead? Something like this in pkg/controller/alias.go:

package controller

import "github.com/scaleway/cluster-api-provider-scaleway/internal/controller"

// Following variables provide access to reconcilers implemented in internal/controller.
var (
	NewScalewayClusterReconciler             = controller.NewScalewayClusterReconciler
	NewScalewayMachineReconciler             = controller.NewScalewayMachineReconciler
	NewScalewayManagedClusterReconciler      = controller.NewScalewayManagedClusterReconciler
	NewScalewayManagedControlPlaneReconciler = controller.NewScalewayManagedControlPlaneReconciler
	NewScalewayManagedMachinePoolReconciler  = controller.NewScalewayManagedMachinePoolReconciler
)

The cluster-api project does something similar for their controllers package: https://github.com/kubernetes-sigs/cluster-api/blob/main/controllers/alias.go

Just tested it, works as expected, I've updated the PR.